-
Notifications
You must be signed in to change notification settings - Fork 153
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Enable stylecheck linter #2505
Enable stylecheck linter #2505
Conversation
Co-authored-by: Vivek Singh <vsingh.ggits.2010@gmail.com>
@@ -12,6 +12,7 @@ | |||
// See the License for the specific language governing permissions and | |||
// limitations under the License. | |||
|
|||
// Package repositoryserver validates the secret credentials for all location profiles. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[question] Do we have a lint checker for comment on package name? I don't think that is necessary
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, it is. ST1000(stylecheck linter). It says we should describe package about what its doing in any one file atleast. Although I have disabled this ST1000
in this PR, but had already made this change, so decided to keep the description as it is and later on ST1000
will be enabled as a part of another PR will add similar description for other packages as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@PrasadG193 Checked kubernates repository. Found few examples where this linter is handled:
- https://github.com/kubernetes/kubernetes/blob/0fc167103189a4d462c3cc7a17b360c3d998f4bf/cmd/kube-apiserver/app/options/options.go#L17
- https://github.com/kubernetes/kubernetes/blob/0fc167103189a4d462c3cc7a17b360c3d998f4bf/cmd/kube-apiserver/app/aggregator.go#L17
- https://github.com/kubernetes/kubernetes/blob/0fc167103189a4d462c3cc7a17b360c3d998f4bf/cmd/kube-apiserver/apiserver.go#L17
@saima-s let's merge 🚀 it. |
Will merge. |
Change Overview
This PR enables stylecheck golang linter
Pull request type
Please check the type of change your PR introduces:
Issues
Test Plan